???.methods.feedback - Code Metrics - diglabby/doika - Measure and Improve Code Quality continuously with Scrutinizer

???.methods.feedback A
last analyzed 2019-03-20 16:51 UTC

↳ Parent: resources/js/dashboard/mixins/form.js

Complexity

Conditions	2
Paths	2

Size

Total Lines	5
Code Lines	3

Duplication

Lines	5
Ratio	100 %

Importance

Changes

Metric	Value
eloc	3
nc	2
nop	1
dl	5
loc	5
c	0
b	0
f	0
cc	2
rs	10

import axios from 'axios'


export default {
  props: ['id'],
  data() {
    return {
      validation: {},
      pending: false
    }
  },
  computed: {
    isNew() {
      return this.id === undefined
    }
  },
  methods: {
    async fetchData() {
      if (!this.isNew) {
        let { data } = await axios.get(
          this.$app.route(`admin.${this.resourceRoute}.show`, {
            [this.modelName]: this.id
          })
        )

        Object.keys(data).forEach(key => {
          if (key in this.model) {
            this.model[key] = data[key]
          }
        })
        this.onModelChanged()
      }
    },
    onModelChanged() {},
    feedback(name) {
      if (this.state(name)) {
function isBig(a) {
    if (a > 5000) {
        return "yes";
    }
}

console.log(isBig(5001)); //returns yes
console.log(isBig(42)); //returns undefined
        return this.validation.errors[name][0]
      }
    },
    state(name) {
      return this.validation.errors !== undefined &&
        this.validation.errors.hasOwnProperty(name)
        ? 'invalid'
        : null
    },
    async onSubmit() {
      this.pending = true
      let router = this.$router
      let action = this.isNew
        ? this.$app.route(`admin.${this.resourceRoute}.store`)
        : this.$app.route(`admin.${this.resourceRoute}.update`, {
            [this.modelName]: this.id
          })

      let formData = this.$app.objectToFormData(this.model)

      if (!this.isNew) {
        formData.append('_method', 'PATCH')
      }

      try {
        let { data } = await axios.post(action, formData)
        this.pending = false

        this.$app.noty[data.status](data.message)
        if (this.listPath) {
          router.push(this.listPath)
        }
      } catch (e) {
        this.pending = false

        // Validation errors
        if (e.response.status === 422) {
          this.validation = e.response.data
          return
        }

        this.$app.error(e)
      }
    }
  },
  created() {
    this.fetchData()
  }
}


1	View Code Duplication	import axios from 'axios'
		0 ignored issues – show Duplication introduced 2019-03-17 18:35 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
2
3		export default {
4		props: ['id'],
5		data() {
6		return {
7		validation: {},
8		pending: false
9		}
10		},
11		computed: {
12		isNew() {
13		return this.id === undefined
14		}
15		},
16		methods: {
17		async fetchData() {
18		if (!this.isNew) {
19		let { data } = await axios.get(
20		this.$app.route(`admin.${this.resourceRoute}.show`, {
21		[this.modelName]: this.id
22		})
23		)
24
25		Object.keys(data).forEach(key => {
26		if (key in this.model) {
27		this.model[key] = data[key]
28		}
29		})
30		this.onModelChanged()
31		}
32		},
33		onModelChanged() {},
34		feedback(name) {
35		if (this.state(name)) {
		0 ignored issues – show Complexity Best Practice introduced 2019-03-17 18:35 UTC by Report Bug Copy Issue Report There is no return statement if `this.state(name)` is `false`. Are you sure this is correct? If so, consider adding `return;` explicitly. This check looks for functions where a `return` statement is found in some execution paths, but not in all. Consider this little piece of code function isBig(a) { if (a > 5000) { return "yes"; } } console.log(isBig(5001)); //returns yes console.log(isBig(42)); //returns undefined The function `isBig` will only return a specific value when its parameter is bigger than 5000. In any other case, it will implicitly return `undefined`. This behaviour may not be what you had intended. In any case, you can add a `return undefined` to the other execution path to make the return value explicit. Loading history...
36		return this.validation.errors[name][0]
37		}
38		},
39		state(name) {
40		return this.validation.errors !== undefined &&
41		this.validation.errors.hasOwnProperty(name)
42		? 'invalid'
43		: null
44		},
45		async onSubmit() {
46		this.pending = true
47		let router = this.$router
48		let action = this.isNew
49		? this.$app.route(`admin.${this.resourceRoute}.store`)
50		: this.$app.route(`admin.${this.resourceRoute}.update`, {
51		[this.modelName]: this.id
52		})
53
54		let formData = this.$app.objectToFormData(this.model)
55
56		if (!this.isNew) {
57		formData.append('_method', 'PATCH')
58		}
59
60		try {
61		let { data } = await axios.post(action, formData)
62		this.pending = false
63
64		this.$app.noty[data.status](data.message)
65		if (this.listPath) {
66		router.push(this.listPath)
67		}
68		} catch (e) {
69		this.pending = false
70
71		// Validation errors
72		if (e.response.status === 422) {
73		this.validation = e.response.data
74		return
75		}
76
77		this.$app.error(e)
78		}
79		}
80		},
81		created() {
82		this.fetchData()
83		}
84		}
85

diglabby / doika

???.methods.feedback A last analyzed 2019-03-20 16:51 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

???.methods.feedback A
last analyzed 2019-03-20 16:51 UTC